test(server-nestjs): cas limites des utilitaires partagés (edges, BigInt, cron) - #2588
Draft
shikanime wants to merge 3 commits into
Draft
test(server-nestjs): cas limites des utilitaires partagés (edges, BigInt, cron)#2588shikanime wants to merge 3 commits into
shikanime wants to merge 3 commits into
Conversation
Open
10 tasks
![github-code-quality[bot]](https://avatars.githubusercontent.com/u/9919?s=60&v=4){kind=small}
shikanime
changed the title
shikanime
force-pushed
the
test/adv-utils
branch
from
August 26, 2026 09:47
ded8651 to
1cff6cd
Compare
shikanime
commented
Aug 28, 2026
shikanime
force-pushed
the
test/adv-utils
branch
3 times, most recently
from
August 31, 2026 11:05
e3e662c to
eb97e9a
Compare
…ls specs Drop as never casts in config.utils flag-strictness tests (parse accepts unknown) and in project-queries specs by using generateProjectCreateInput with faker data and the Prisma payload type for findFirst, per server-nestjs vitest convention. Signed-off-by: William Phetsinorath <william.phetsinorath-open@interieur.gouv.fr> Change-Id: Id26640977d08fdc24155d60cbde460a06a6a6964
shikanime
force-pushed
the
test/adv-utils
branch
2 times, most recently
from
August 31, 2026 12:51
684920d to
c007915
Compare
shikanime
force-pushed
the
test/adv-utils
branch
from
August 31, 2026 12:52
c007915 to
562a1fc
Compare
|
0 New Issues
0 Fixed Issues
0 Accepted Issues
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Issues liées
#2574
Quel est le comportement actuel ?
Les utilitaires partagés avaient des angles morts mesurés :
utils/http.utils.ts46%,config/config.utils.ts59%,project-members.utils0%,vault.utils40%.Quel est le nouveau comportement ?
+495 lignes / ~60 tests sur 5 fichiers :
?Quartz en day-of-month est accepté et documenté comme divergence assumée (Harbor l'utilise) ;138/138 tests verts sur Node 24. Deux comportements surprenants sont scellés par des tests avec commentaires
ponytail-bug:pointant les lignes concernées.Cette PR introduit-elle un breaking change ?
Non.